[latex] Add command problem_slides #413
Conversation
TODO: make problem_slides work for a single problem
| ) | ||
| success &= latex.build_contest_pdfs( | ||
| contest, problems, tmpdir, statement_language, solutions=True, web=True | ||
| success &= latex.build_contest_pdf( |
There was a problem hiding this comment.
Should this (the no_solutions flag) be more granular? It's annoying if things fail because no problem slides are present.
Or maybe better is to never make it an error if problem slides are missing? (But that's kinda tricky when you want slides for all problems?)
Also I checked an doing bt problem_slides at the contest level with some problem-slide.en.tex missing crashes in latex (because file not found). We should probably detect when there's not a single problem-slide present and then just Log that (without error). When only some files are present, it's fine to error IMO.
At the problem level it gives a nice message problem-slide.en.tex not found, so there all is good.
Otherwise LGTM! Thanks for making the PR :)
There was a problem hiding this comment.
That's a good point! Will reduce some errors to warnings today or tomorrow 😄
There was a problem hiding this comment.
I've added some extra checks 🙂 The behaviour at the problem level hasn't changed, this is the new behaviour at the contest level:
- When no
problem-slide.*.texare present:bt problem_slideswill warn for all problems, followed by a failing LaTeX compilation (because the user is explicitly trying to compile them)bt zipwill log "No problem has problem-slide.*.tex, skipping problem slides"
- When at least one
problem-slide.*.texis present:- Both
bt problem_slidesandbt zipwill warn for the problems that do not have it, but will compile the rest
- Both
There was a problem hiding this comment.
That sounds good. Thanks! Merging now.
| ) | ||
| success &= latex.build_contest_pdfs( | ||
| contest, problems, tmpdir, statement_language, solutions=True, web=True | ||
| success &= latex.build_contest_pdf( |
There was a problem hiding this comment.
That sounds good. Thanks! Merging now.
At long last, here's the final PR for
bt problem_slides, which has been used successfully for the live streams of the past three NWERCs now 😄The
bt problem_slidescommand compiles a slide deck with one slide per problem, each slide containing a summary of the problem that can be used to show during the live stream of a contest. For an example, see https://2024.nwerc.eu/main/problem-slides.pdf.Note that this adds
skel/problem/problem_statement/problem-slide.en.tex. However, the majority of contests will not have a live stream. I guess this is fine though, because the file can be safely deleted and/or be left out in a customskel/problemtree.I'd like someone to double-check before merging, perhaps try to run the command on the NWERC repo again (both on the problem and contest levels), just in case. Only the first commit was actually used during the past NWERC, I've added the last three commits only today to wrap things up 🙂